Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update naga to e98bd92 #3352

Merged
merged 2 commits into from
Jan 10, 2023
Merged

Update naga to e98bd92 #3352

merged 2 commits into from
Jan 10, 2023

Conversation

nical
Copy link
Contributor

@nical nical commented Jan 5, 2023

Description

A routine naga update update.
I don't think there is anything to add in the changelog since naga has its own.

@teoxoy
Copy link
Member

teoxoy commented Jan 9, 2023

This test now passes

.specific_failure(Some(wgpu::Backends::GL), None, Some("ANGLE"), false)

not sure why though (new version of angle?)

cc @cwfitzgerald (since it was added quite recently in #3296)

@cwfitzgerald
Copy link
Member

I don't know why either - it was failing due to validation errors stemming from #3349 and it's not like that has been fixed.

@teoxoy
Copy link
Member

teoxoy commented Jan 10, 2023

What should we do for now to get this merged? I reran the workflow and it still succeeds when it shouldn't.
Comment out the failure and go from there?

nical added 2 commits January 10, 2023 17:51
We aren't quite sure why/how the latest naga changes fixed it though.
Copy link
Member

@cwfitzgerald cwfitzgerald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright, this is fine to keep things moving, it doesn't make any sense though - will look into it tomorrow.

@cwfitzgerald cwfitzgerald merged commit 5a2e60c into gfx-rs:master Jan 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants